Skip to content

feat: add optional boxShadow to ChatBubble#434

Merged
vatsaltanna-simformsolutions merged 3 commits into
SimformSolutionsPvtLtd:mainfrom
torretorich:feature/chat-bubble-box-shadow
Jun 4, 2026
Merged

feat: add optional boxShadow to ChatBubble#434
vatsaltanna-simformsolutions merged 3 commits into
SimformSolutionsPvtLtd:mainfrom
torretorich:feature/chat-bubble-box-shadow

Conversation

@torretorich

Copy link
Copy Markdown
Contributor

Description

Adds an optional boxShadow field to ChatBubble and wires it into TextMessageView’s BoxDecoration for outgoing and incoming bubbles when set.

Existing behavior: Text bubbles only supported color, borderRadius, border, padding/margin, etc.; there was no supported way to add a drop shadow without forking.

This PR: When boxShadow is null (default), appearance is unchanged. When set, the configured List<BoxShadow> is applied to the text message bubble.

Motivation: Allows apps to add subtle elevation/shadow for readability and visual hierarchy without patching the package.

Checklist

  • The title of my PR starts with a [Conventional Commit] prefix (fix:, feat:, docs: etc).
  • I have followed the [Contributor Guide] when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

Related Issues

@torretorich torretorich changed the title Feature/chat bubble box shadow feat: add optional boxShadow to ChatBubble Apr 9, 2026

@vasu-nageshri vasu-nageshri left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vatsaltanna-simformsolutions vatsaltanna-simformsolutions merged commit a5b0cd6 into SimformSolutionsPvtLtd:main Jun 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants